apps/docs/src/app/layout.config.tsx (1)

17-19: ⚡ Quick win

Extract repeated logo dimensions into named constants.

Line 17, Line 19, Line 24, and Line 26 duplicate intrinsic dimension literals (804, 2585). A shared constant pair keeps the light/dark variants in sync and makes intent clearer.

♻️ Proposed refactor

+const LOGO_INTRINSIC_WIDTH = 2585;
+const LOGO_INTRINSIC_HEIGHT = 804;
+
 export const baseOptions = {
   nav: {
     title: (
       <>
         <Image
           alt="ProofKit"
           className="block h-auto w-[96px] dark:hidden"
-          height={804}
+          height={LOGO_INTRINSIC_HEIGHT}
           src="/logo-horiz-light.svg"
-          width={2585}
+          width={LOGO_INTRINSIC_WIDTH}
         />
         <Image
           alt="ProofKit"
           className="hidden h-auto w-[96px] dark:block"
-          height={804}
+          height={LOGO_INTRINSIC_HEIGHT}
           src="/logo-horiz-dark.svg"
-          width={2585}
+          width={LOGO_INTRINSIC_WIDTH}
         />
       </>
     ),

As per coding guidelines, **/*.{js,jsx,ts,tsx}: Use meaningful variable names instead of magic numbers - extract constants with descriptive names.

Also applies to: 24-26

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@apps/docs/src/app/layout.config.tsx` around lines 17 - 19, Extract the
repeated intrinsic dimensions into descriptive constants (e.g., LOGO_WIDTH and
LOGO_HEIGHT) and replace the magic numbers used for both logo variants (the
Image/Image-like props with src "/logo-horiz-light.svg" and
"/logo-horiz-dark.svg") so both the light and dark logo usages reference the
same constants; update any imports/exports or nearby scope to declare these
constants near the top of the module (layout.config.tsx) so the height and width
props use LOGO_HEIGHT and LOGO_WIDTH instead of 804 and 2585.